Skip to content

[New] 新增报名比赛 - #4

Merged
Transparent-fish merged 5 commits into
mainfrom
dev
Jul 12, 2026
Merged

[New] 新增报名比赛#4
Transparent-fish merged 5 commits into
mainfrom
dev

Conversation

@Transparent-fish

Copy link
Copy Markdown
Owner

No description provided.

@github-actions

Copy link
Copy Markdown

PR 审查报告

检查结果

检查项 状态
pnpm build
pnpm lint
pnpm test
函数 <= 130 行

未通过的检查

全部通过,无需修改。


自动生成 by PR Review workflow

@Transparent-fish

Copy link
Copy Markdown
Owner Author

/gemini review

@Transparent-fish Transparent-fish mentioned this pull request Jul 12, 2026
3 tasks

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request restructures the VS Code extension by modularizing network requests and translation utilities into dedicated files under a new tools directory, introducing a new contest registration feature (SignUpContest.ts), and adding a project specification document (docs/Standard.md). The feedback highlights critical improvements: resolving relative redirect URLs to absolute paths in the fetch utility, adding a Content-Length header to POST requests to prevent potential 411 errors, and parallelizing translation requests with Promise.all to boost performance. Additionally, the registration success detection should be made more robust against redirects, and several instances of the PascalCase variable Rated should be renamed to camelCase (supportsRated) to comply with the newly established project guidelines.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

Comment thread apps/vscode-extension/src/SignUpContest.ts Outdated
Comment thread apps/vscode-extension/src/tools/fetch.ts
Comment thread apps/vscode-extension/src/SignUpContest.ts
Comment thread apps/vscode-extension/src/SignUpContest.ts
Comment thread apps/vscode-extension/src/tools/fetch.ts
Comment thread apps/vscode-extension/src/extension.ts
Comment thread packages/webview/src/WebviewApp.tsx
Comment thread packages/webview/src/types.ts
@github-actions

Copy link
Copy Markdown

PR 审查报告

检查结果

检查项 状态
pnpm build
pnpm lint
pnpm test
函数 <= 130 行

未通过的检查

全部通过,无需修改。


自动生成 by PR Review workflow

@Transparent-fish

Copy link
Copy Markdown
Owner Author

done.

@Transparent-fish
Transparent-fish merged commit 7a9f76d into main Jul 12, 2026
1 check passed
@Transparent-fish Transparent-fish mentioned this pull request Jul 15, 2026
8 tasks
@Transparent-fish Transparent-fish moved this to Done in Phase 1 Jul 21, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant